Skip to content

Conversation

lijinpei2008
Copy link
Contributor

…->v4

Description

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

@lijinpei2008
Copy link
Contributor Author

/azp run

Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@lijinpei2008
Copy link
Contributor Author

/azp run

Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@lijinpei2008 lijinpei2008 added the Breaking Change Release This PR contains breaking change label Aug 18, 2025
Copy link

To the author of the pull request,
This PR was labeled "Breaking Change Release" because it contains breaking changes.

  • According to our policy, breaking changes can only take place during major release and they must be preannounced.
  • Please follow our guide on the detailed steps.
  • Required: Please fill in the task below to facilitate our contact,you will receive notifications related to breaking changes.

@lijinpei2008 lijinpei2008 marked this pull request as ready for review August 22, 2025 10:33
@Copilot Copilot AI review requested due to automatic review settings August 22, 2025 10:33
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the AutoRest generation tool for the Az.Attestation module from version 3 to version 4, which introduces several changes to the generated PowerShell cmdlets and their documentation.

  • Updates the AutoRest PowerShell generator from version 3.x to 4.x
  • Adds new JSON parameter sets for Create and Update cmdlets to support file-based and string-based JSON input
  • Refreshes all generated documentation and help files with updated parameter sets and type references

Reviewed Changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tools/StaticAnalysis/Exceptions/Az.Attestation/BreakingChangeIssues.csv Adds exception entries for breaking changes in property types and parameter sets resulting from the AutoRest upgrade
src/Attestation/Attestation/help/*.md Updates help documentation with new parameter sets and corrected type references
src/Attestation/Attestation.Autorest/docs/*.md Updates AutoRest-generated documentation with new parameter sets and corrected type references
src/Attestation/Attestation.Autorest/custom/New-AzAttestationProvider.ps1 Updates cmdlet implementation with new parameter sets and corrected type references
src/Attestation/Attestation.Autorest/custom/JsonWebKeyHelper.cs Updates namespace and collection type usage to match new AutoRest generation patterns
src/Attestation/Attestation.Autorest/README.md Updates AutoRest configuration to use version 4.x and removes deprecated directives

@@ -3,7 +3,7 @@
#
# Generated by: Microsoft Corporation
#
# Generated on: 2025-07-27
# Generated on: 2025-07-14
Copy link
Preview

Copilot AI Aug 22, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The generated date has moved backwards from 2025-07-27 to 2025-07-14, which suggests an incorrect timestamp during regeneration. This should be updated to reflect the actual generation date.

Suggested change
# Generated on: 2025-07-14
# Generated on: 2025-07-27

Copilot uses AI. Check for mistakes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaking Change Release This PR contains breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant